Jmercer/complete plugin sync fix #6527
Open
+113
−23
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fix plugin documentation sync automation
Summary
This PR fixes critical issues in the plugin documentation sync workflow that prevented it from running end-to-end. The workflow can now successfully authenticate across repositories, locate source files, transform plugin documentation, and build the Hugo site.
Problems Solved
1. Cross-Repository Authentication
Issue: Default
GITHUB_TOKENlacks permissions to access files from other repositories, causing 403 errors when attempting to cloneinfluxdb3_pluginsfrom thedocs-v2workflow.2. Incorrect Path Resolution in Transformation Script
Issue: The transformation script (
port_to_docs.js) reads source paths fromdocs_mapping.yaml, not from theINFLUXDB3_PLUGINS_PATHenvironment variable. The configured paths used../../.ext/influxdb3_pluginsbut the actual directory structure requires../../../.ext/influxdb3_plugins.Testing
Tested with workflow run using plugin
basic_transformation:Results:
Related Work
This PR builds on:
PLUGINS_CONTENT_READ_TOKENandISSUES_READ_WRITE_TOKEN)Impact
Before this PR:
After this PR:
Future Improvements
docs-v2nesting (would require changes across workflow)How to Test
jmercer/complete-plugin-sync-fixbasic_transformationmainChecklist
cc: @jstirnaman